Skip to content

fix: 채팅 이미지 렌더 URL 정리#527

Merged
manNomi merged 1 commit into
mainfrom
fix/chat-image-clean-render-url
May 26, 2026
Merged

fix: 채팅 이미지 렌더 URL 정리#527
manNomi merged 1 commit into
mainfrom
fix/chat-image-clean-render-url

Conversation

@manNomi
Copy link
Copy Markdown
Contributor

@manNomi manNomi commented May 25, 2026

관련 이슈

  • resolves: #이슈 번호

작업 내용

  • 채팅 이미지 health check probe는 기존처럼 cache-busting URL로 확인하되, 실제 렌더링되는 이미지 URL은 원본 clean URL을 사용하도록 수정했습니다.
  • 최종 <Image> src에 __chat_image_health_check query param이 남지 않게 했습니다.

특이 사항

  • CDN 반영 지연 대응을 위한 health check 재시도 흐름은 유지합니다.
  • sender 로컬 preview(blob:/data:) 동작은 그대로 유지됩니다.

리뷰 요구사항 (선택)

  • health check probe와 실제 렌더 URL 분리가 의도대로 동작하는지 확인 부탁드립니다.

검증

  • pnpm --filter @solid-connect/web lint:check
  • pnpm --filter @solid-connect/web typecheck
  • commit hook CI parity checks passed
  • push hook CI parity checks passed

@manNomi manNomi requested review from enunsnv and wibaek as code owners May 25, 2026 09:57
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

Warning

Review limit reached

@manNomi, we couldn't start this review because you've used your available PR reviews for now.

Your plan includes 1 review of capacity. Refill in 32 minutes and 38 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b30627fe-5e5f-4449-89b6-1a487fd5a532

📥 Commits

Reviewing files that changed from the base of the PR and between 46a58d9 and dfad52f.

📒 Files selected for processing (1)
  • apps/web/src/app/mentor/chat/[chatId]/_ui/ChatContent/_ui/ChatMessageBox/index.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/chat-image-clean-render-url

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connection-web Ready Ready Preview, Comment May 25, 2026 9:59am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped May 25, 2026 9:59am

@github-actions github-actions Bot added the web label May 25, 2026
@manNomi manNomi merged commit 07003eb into main May 26, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wibaek wibaek Awaiting requested review from wibaek wibaek is a code owner

@enunsnv enunsnv Awaiting requested review from enunsnv enunsnv is a code owner

Assignees

No one assigned

Labels

web

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@manNomi